-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
DOC: Added sentence to show that unmatched values return -1. #43738
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Hello @DhruvBShetty! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found: There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻 Comment last updated at 2021-11-01 12:05:03 UTC |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @DhruvBShetty
pandas/core/indexes/base.py
Outdated
@@ -3484,6 +3484,9 @@ def get_loc(self, key, method=None, tolerance=None): | |||
indexer should be then used as an input to ndarray.take to align the | |||
current data to the new index. | |||
|
|||
Returns -1 for unmatched values, for further explanation see the example |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
move to the Notes section
This pull request is stale because it has been open for thirty days with no activity. Please update or respond to this comment if you're still interested in working on this. |
Hi @DhruvBShetty can you address the comment please |
Interested in finishing this. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please check the pandas docstring guide for how to write the "notes" section
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you run the validating docstrings script for this function?
Left a line between %(raises_section)s and notes accidentally. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @DhruvBShetty |
In [1]: python make.py --single pandas.Index.get_indexer
Out [1]:
In [2]: pre-commit run --files pandas/core/indexes/base.py
Out [2] :